Skip to content

Refactor LCAO code Step 3 (Useful to know how to do refactoring in order to remove UHM, which was a "global" class used everywhere in LCAO code)#3914

Merged
mohanchen merged 62 commits into
deepmodeling:developfrom
mohanchen:new
Apr 4, 2024
Merged

Refactor LCAO code Step 3 (Useful to know how to do refactoring in order to remove UHM, which was a "global" class used everywhere in LCAO code)#3914
mohanchen merged 62 commits into
deepmodeling:developfrom
mohanchen:new

Conversation

@mohanchen

Copy link
Copy Markdown
Collaborator

Reminder

  • Have you linked an issue with this pull request?
  • Have you added adequate unit tests and/or case tests for your pull request?
  • Have you noticed possible changes of behavior below or in the linked issue?
  • Have you explained the changes of codes in core modules of ESolver, HSolver, ElecState, Hamilt, Operator or Psi? (ignore if not applicable)

Linked Issue

Fix #3912

Unit Tests and/or Case Tests for my changes

  • A unit test is added for each new feature or bug fix.

What's changed?

  • Example: My changes might affect the performance of the application under certain conditions, and I have tested the impact on various scenarios...

Any changes of core modules? (ignore if not applicable)

  • Example: I have added a new virtual function in the esolver base class in order to ...

@mohanchen mohanchen requested a review from jinzx10 April 3, 2024 05:33
@mohanchen mohanchen merged commit 8bee71b into deepmodeling:develop Apr 4, 2024
@mohanchen mohanchen self-assigned this May 8, 2024
@mohanchen mohanchen added the Useful Information Useful information for others to learn/study label May 8, 2024
@mohanchen mohanchen changed the title Refactor LCAO code Step 3 Refactor LCAO code Step 3 (Useful to know how to do refactoring in order to remove UHM, which is a "global" class used everywhere in LCAO code) May 8, 2024
@mohanchen mohanchen changed the title Refactor LCAO code Step 3 (Useful to know how to do refactoring in order to remove UHM, which is a "global" class used everywhere in LCAO code) Refactor LCAO code Step 3 (Useful to know how to do refactoring in order to remove UHM, which was a "global" class used everywhere in LCAO code) May 8, 2024
@mohanchen mohanchen added the The Absolute Zero Reduce the "entropy" of the code to 0 label May 12, 2024
@mohanchen mohanchen deleted the new branch July 26, 2024 13:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

The Absolute Zero Reduce the "entropy" of the code to 0 Useful Information Useful information for others to learn/study

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Refactor LCAO code Step 3

2 participants